-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[VEN-2770] Revoke Guardian Permissions #363
Conversation
vips/vip-355/bscmainnet.ts
Outdated
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [arbitrumone.REDSTONE_ORACLE, "setTokenConfig(TokenConfig)", arbitrumone.GUARDIAN], | ||
dstChainId: LzChainId.arbitrumone, | ||
}, | ||
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [arbitrumone.REDSTONE_ORACLE, "setDirectPrice(address,uint256)", arbitrumone.GUARDIAN], | ||
dstChainId: LzChainId.arbitrumone, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't revoke these permissions. They are granted in the ChainlinkOracle, that is similar to this oracle
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
{ | ||
target: ETHEREUM_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ethereum.REDSTONE_ORACLE, "setTokenConfig(TokenConfig)", ethereum.GUARDIAN], | ||
dstChainId: LzChainId.ethereum, | ||
}, | ||
{ | ||
target: ETHEREUM_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ethereum.REDSTONE_ORACLE, "setDirectPrice(address,uint256)", ethereum.GUARDIAN], | ||
dstChainId: LzChainId.ethereum, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't revoke these permissions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
}; | ||
return makeProposal( | ||
[ | ||
// Revoke Permissions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
}; | ||
return makeProposal( | ||
[ | ||
// Revoke Permissions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The zero address was used to grant permissions on the IR models. So, I suppose we shouldn't use specific addresses to revoke these permissions, but the zero address
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ARBITRUM_ONE_PRIME, "setTokensDistributionSpeed(address[],uint256[])", arbitrumone.GUARDIAN], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
params: [ARBITRUM_ONE_PRIME, "setTokensDistributionSpeed(address[],uint256[])", arbitrumone.GUARDIAN], | |
params: [ARBITRUM_ONE_PLP, "setTokensDistributionSpeed(address[],uint256[])", arbitrumone.GUARDIAN], |
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ARBITRUM_ONE_PRIME, "setMaxTokensDistributionSpeed(address[],uint256[])", arbitrumone.GUARDIAN], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
params: [ARBITRUM_ONE_PRIME, "setMaxTokensDistributionSpeed(address[],uint256[])", arbitrumone.GUARDIAN], | |
params: [ARBITRUM_ONE_PLP, "setMaxTokensDistributionSpeed(address[],uint256[])", arbitrumone.GUARDIAN], |
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ARBITRUM_ONE_PRIME, "setMaxLoopsLimit(uint256)", arbitrumone.GUARDIAN], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
params: [ARBITRUM_ONE_PRIME, "setMaxLoopsLimit(uint256)", arbitrumone.GUARDIAN], | |
params: [ARBITRUM_ONE_PLP, "setMaxLoopsLimit(uint256)", arbitrumone.GUARDIAN], |
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ARBITRUM_ONE_PLP, "updateAlpha(uint128,uint128)", arbitrumone.GUARDIAN], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
params: [ARBITRUM_ONE_PLP, "updateAlpha(uint128,uint128)", arbitrumone.GUARDIAN], | |
params: [ARBITRUM_ONE_PRIME, "updateAlpha(uint128,uint128)", arbitrumone.GUARDIAN], |
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ARBITRUM_ONE_PLP, "updateMultipliers(address,uint256,uint256)", arbitrumone.GUARDIAN], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
params: [ARBITRUM_ONE_PLP, "updateMultipliers(address,uint256,uint256)", arbitrumone.GUARDIAN], | |
params: [ARBITRUM_ONE_PRIME, "updateMultipliers(address,uint256,uint256)", arbitrumone.GUARDIAN], |
{ | ||
target: ARBITRUM_ONE_ACM, | ||
signature: "revokeCallPermission(address,string,address)", | ||
params: [ARBITRUM_ONE_PLP, "setStakedAt(address[],uint256[])", arbitrumone.GUARDIAN], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
params: [ARBITRUM_ONE_PLP, "setStakedAt(address[],uint256[])", arbitrumone.GUARDIAN], | |
params: [ARBITRUM_ONE_PRIME, "setStakedAt(address[],uint256[])", arbitrumone.GUARDIAN], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PLP should be replaced by prime and vice versa on many places
Closing this PR and will open new PR utilising ACMCommandsAggregator for revoking permissions |
Description
Resolves VEN